Skip to content

Mod/dev 14599 Tanstack Query for Adv Search Results Table#5195

Merged
nick-torres merged 27 commits into
qatfrom
mod/DEV-14599
May 22, 2026
Merged

Mod/dev 14599 Tanstack Query for Adv Search Results Table#5195
nick-torres merged 27 commits into
qatfrom
mod/DEV-14599

Conversation

@jagu65

@jagu65 jagu65 commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Description:
Refactored ResultsTableContainer to move parameter preparation and data fetching to a custom hook. The data fetching is now utilizing tanstack query. Also refactored ResultsView in order to utilize tanstack query as well and pass the awards counts down to results table.

JIRA Ticket:
DEV-14599

Mockup:
n/a

The following are ALL required for the PR to be merged:

Author:

  • Linked to this PR in JIRA ticket
  • Scheduled and completed design review
  • Provided instructions for testing in JIRA ticket and PR if applicable
  • Verified cross-browser compatibility: Chrome, Safari, Firefox, Edge
  • Verified mobile/tablet/desktop/monitor responsiveness
  • Verified that this PR does not create any new accessibility issues (via Axe Chrome extension or Lighthouse report)
  • Added Unit Tests for helper functions, reducers, models and Container/Component Interactivity Expectations if applicable React Testing Library
  • API contract updated if applicable

@nick-torres nick-torres self-assigned this Apr 1, 2026
@nick-torres nick-torres added do not merge pending pr merging of another usaspending-website pr is needed before merging labels Apr 1, 2026
@jagu65 jagu65 removed do not merge pending pr merging of another usaspending-website pr is needed before merging labels Apr 14, 2026
@jagu65 jagu65 changed the title Mod/dev 14599 Mod/dev 14599 Tanstack Query for Adv Search Results Table Apr 16, 2026
@nick-torres

Copy link
Copy Markdown
Contributor

conflicts + 422 error when switching to subawards bc it's attempting to sort on award amount but that field doesn't exist for subawards

@jagu65

jagu65 commented Apr 20, 2026

Copy link
Copy Markdown
Contributor Author

conflicts + 422 error when switching to subawards bc it's attempting to sort on award amount but that field doesn't exist for subawards

Fixed!

@anjenkin anjenkin added do not merge pending sprint start to be merged after code freeze ends, at the start of a new sprint labels Apr 23, 2026
@jagu65 jagu65 removed the pending sprint start to be merged after code freeze ends, at the start of a new sprint label May 12, 2026
@anjenkin anjenkin added the pending sprint start to be merged after code freeze ends, at the start of a new sprint label May 14, 2026
@jagu65 jagu65 removed do not merge pending sprint start to be merged after code freeze ends, at the start of a new sprint labels May 21, 2026
@nick-torres nick-torres merged commit a53b091 into qat May 22, 2026
6 checks passed
@nick-torres nick-torres deleted the mod/DEV-14599 branch May 22, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants